-
Notifications
You must be signed in to change notification settings - Fork 204
[8.x] Don't modify index mappings for Alerts indices #6882
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
A documentation preview will be available soon. Request a new doc build by commenting
If your PR continues to fail for an unknown reason, the doc build pipeline may be broken. Elastic employees can check the pipeline status here. |
…ndices (#1735) Partially addresses elastic/security-team#12799. Inserts a note letting users know that they should not modify alert index mappings. Previews: - [Alert schema](https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/1735/reference/security/fields-and-object-schemas/alert-schema) - [About detection rules | Data views and index patterns](https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/1735/solutions/security/detect-and-alert/about-detection-rules#views-index-patterns) - [Query alert indices](https://docs-v3-preview.elastic.dev/elastic/docs-content/pull/1735/solutions/security/detect-and-alert/query-alert-indices) Corresponding 8.x PR: elastic/security-docs#6882
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you!
Note to self: Before merge this PR, need to link to docs for adding runtime fields to rules. Will need to make this change to the 9.x/Serverless docs as well. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gracias, gracias!
Made 8.x changes in this PR and 9.x/Serverless changes in elastic/docs-content#1957. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One slight suggestion; otherwise good to go!
Co-authored-by: Janeen Mikell Roberts <[email protected]>
…ndices pt.2 (#1957) Related to elastic/security-team#12799. Updates note that lets users know that they should not modify alert index mappings. Redirects them to runtime fields instead. Corresponding 8.x PR: elastic/security-docs#6882
* First draft * Add link to runtime fields * apply to other areas * remove s * cleanup * Update docs/reference/alert-schema.asciidoc Co-authored-by: Janeen Mikell Roberts <[email protected]> --------- Co-authored-by: Janeen Mikell Roberts <[email protected]> (cherry picked from commit a09754a)
* First draft * Add link to runtime fields * apply to other areas * remove s * cleanup * Update docs/reference/alert-schema.asciidoc Co-authored-by: Janeen Mikell Roberts <[email protected]> --------- Co-authored-by: Janeen Mikell Roberts <[email protected]> (cherry picked from commit a09754a)
* First draft * Add link to runtime fields * apply to other areas * remove s * cleanup * Update docs/reference/alert-schema.asciidoc --------- (cherry picked from commit a09754a) Co-authored-by: Nastasha Solomon <[email protected]> Co-authored-by: Janeen Mikell Roberts <[email protected]>
* First draft * Add link to runtime fields * apply to other areas * remove s * cleanup * Update docs/reference/alert-schema.asciidoc --------- (cherry picked from commit a09754a) Co-authored-by: Nastasha Solomon <[email protected]> Co-authored-by: Janeen Mikell Roberts <[email protected]>
Partially addresses https://github.com/elastic/security-team/issues/12799. Inserts a note letting users know that they should not modify alert index mappings.
Previews:
Corresponding 9.x/Serverless PR: elastic/docs-content#1735 and elastic/docs-content#1957